Skip to content

Feature: save as new, duplicate#171

Closed
idjohnson wants to merge 2 commits intostan-smith:masterfrom
idjohnson:save-as-new
Closed

Feature: save as new, duplicate#171
idjohnson wants to merge 2 commits intostan-smith:masterfrom
idjohnson:save-as-new

Conversation

@idjohnson
Copy link
Copy Markdown

Description

I often want to show multiple transforms of an idea, or a migration/build out of an idea. To date, I have to export a JSON, do new, and import the JSON which is a hassle. This allows us to now duplicate a server diagram to a new "(copy)" file as well as saving the existing session as a new ID (instead of overwriting the current copy)

I used Gemini 3 Pro to implement (in case you have issues with AI generated code, though I didn't see anything in CONTRIBUTING.md about that)

Related Issue

Fixes #123

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Performance improvement

Testing

  • Tests pass locally
  • Added new tests
  • Manual testing completed

Screenshots (if applicable)

image image

@stan-smith
Copy link
Copy Markdown
Owner

hey @idjohnson thanks for submitting this PR, thanks for being open about the AI generated code.
Not a problem, just obviously we will need to make sure it isn't going to accidentally delete diagrams etc, so I'll need to do some quite extensive testing for this 👍
Cheers, Stan

@idjohnson
Copy link
Copy Markdown
Author

sounds good. I'll be using it on my instance (https://fossflow.tpk.pw/) so I'll do some more real world testing as well.

@stan-smith
Copy link
Copy Markdown
Owner

@idjohnson awesome, appreciate it, hope you're the new features since you last looked!
I'll hopefully get it up and running tomorrow, busy weekend.
Cheers,
Stan

@stan-smith
Copy link
Copy Markdown
Owner

Hi @idjohnson I am going to be testing this soon, just on holiday at the moment!

@stan-smith
Copy link
Copy Markdown
Owner

Hey @idjohnson would you be comfortable adding some integration tests to see if this works? I have a basic testing infrastructure in /e2e-tests that uses pytest + selenium to interact with the page.
Thanks,
Stan

@stan-smith
Copy link
Copy Markdown
Owner

Hi 👋

Thanks for the contribution. I've recently tightened up the contribution guidelines for FossFLOW, including enforcing Conventional Commits for PR titles.

Your PR title doesn't follow the required format:

<type>(<scope>): <subject>

Valid types: feat, fix, perf, refactor, docs, style, test, chore, build, ci

Examples:

  • feat: add undo/redo functionality
  • fix: prevent menu from opening during drag
  • docs: update installation instructions

I'm closing this PR. If your changes are still relevant, please:

  1. Read the updated CONTRIBUTING.md
  2. Resubmit with a compliant PR title
  3. Make sure you fill in the PR template

Cheers,
Stan

@stan-smith stan-smith closed this Feb 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

blank diagram after load

2 participants